fix(deps): update dependency saxonche to v13 - #3818
Conversation
|
We have in issue with harvest previews in uwsgi context with saxonche v13 (ie try preview this harvester with saxonche v13): We don't encounter any error in the celery worker or with a python interpreter, only in the uwsgi context on dev & demo. |
8b26532 to
5b4ee0e
Compare
5b4ee0e to
92bef94
Compare
92bef94 to
6a91e8b
Compare
Why this fails under uwsgisaxonche 13 reserves 32 GiB of address space when a CSW backend creates its Our uwsgi workers run with It's a native exit, not a Python exception — the whole worker dies, not just the CI is green because no test runs under a memory limit. Added one that starts What supporting 13 would requireBoth our memory guards measure virtual memory, which stopped being a proxy for ; limit-as = 1024 ; drop — kills the worker over reserved-but-unused memory
; reload-on-as = 768 ; drop — would recycle the worker after every Saxon request
reload-on-rss = 192 ; keep — this is the one that tracks real memory
evil-reload-on-rss = 256 ; add — see below
Saxonche v12 vs v1313 gives us nothing: XSLT/XPath 4.0 is PE/EE only, we run HE with an XSLT 1.0 |
|
Test added in #3882, so this PR should become red after merging to main and rebasing it. |
|
That's a pretty crazy change in memory footprint! Probably worth pushing upstream (I can do it if you'd like)?
The SEMIC converter bumped to XSLT 2.0 on main, and an udata upgrade in under way. FYI only since it doesn't impact your conclusions. |
This is not really a change in memory footprint since it's reserved memory and not used (it doesn't even need this physical memory on the computer as I understand it) |
|
It's not active footprint but still, it claims it might use that much, and it would make sense to be able to tune it for the very reason you're hitting. From the GraalVM doc I see I couldn't find much more than this thread https://saxonica.plan.io/boards/4/topics/9976, which claims it's not easy to do in SaxonC :/ There's absolutely no emergency as you explained, but I'm worried we might at some point bump into a security update or something similar that will only be available in version 13. |
6a91e8b to
8594c17
Compare
8594c17 to
75775c2
Compare
This PR contains the following updates:
>=12.8.0,<13.0.0→>=13.0.0,<13.1.0Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.